Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue 1250 #1839

Merged
merged 4 commits into from
Aug 12, 2024
Merged

Issue 1250 #1839

merged 4 commits into from
Aug 12, 2024

Conversation

DevPio
Copy link
Contributor

@DevPio DevPio commented Jun 22, 2024

Q A
Bug fix? [ ]
New feature? [ ]
New sample? [ ]
Related issues? fixes #X, partially #Y, mentioned in #Z

What's in this Pull Request?

#1250

Gravando 2024-06-22 054402

Gravando 2024-06-22 054942

This PR introduces the ability to clear the selection in the SitePicker control when in single selection mode. It includes a custom renderer for the chevron icon, which now displays a "Cancel" icon alongside the chevron when an item is selected. Clicking the "Cancel" icon clears the current selection. The "Cancel" icon is only shown when an item is selected, ensuring a clean UI when there is no selection. These changes improve the user experience by making selection management more intuitive.

@michaelmaillot
Copy link
Collaborator

Hi @DevPio,

Nice improvment, thanks! I think it's ok for this to be merged, but first: would you mind updating your branch to get latest version?

@DevPio
Copy link
Contributor Author

DevPio commented Aug 12, 2024

@michaelmaillot done!

@michaelmaillot michaelmaillot merged commit a3d3ce0 into pnp:dev Aug 12, 2024
1 check passed
@michaelmaillot
Copy link
Collaborator

Merged manually, thank you!

@michaelmaillot michaelmaillot added this to the 3.19.0 milestone Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants